Skip to content

[TorchToTosa] add conv reshape in core lowering#4494

Open
catcor01 wants to merge 4 commits intollvm:mainfrom
catcor01:conv_reshape
Open

[TorchToTosa] add conv reshape in core lowering#4494
catcor01 wants to merge 4 commits intollvm:mainfrom
catcor01:conv_reshape

Conversation

@catcor01
Copy link
Contributor

@catcor01 catcor01 commented Mar 9, 2026

  • Insert rank-4/5 reshapes for conv inputs/weights during TorchToTosa lowering

Change-Id: Ica1b5cc265822ecd054f832908ec31bc2325c661

@catcor01
Copy link
Contributor Author

catcor01 commented Mar 9, 2026

@sahas3 @Lallapallooza

- Insert rank-4/5 reshapes for conv inputs/weights during TorchToTosa lowering

Signed-off-by: Cathal Corbett <cathal.corbett@arm.com>
Change-Id: Ica1b5cc265822ecd054f832908ec31bc2325c661
Change-Id: I5c0c1a5ae2d90cee500dc76247f5952b99bb48f9
Signed-off-by: Cathal Corbett <cathal.corbett@arm.com>
Change-Id: I55752f920f8ad170b3d35c0c8bf5f8b94c4d9de0
Signed-off-by: Cathal Corbett <cathal.corbett@arm.com>
if (tmplIt == argToTemplates.end())
return failure();

const RankTemplate *match = nullptr;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Template matching is rank-only and first-hit. If multiple reshape templates exist for the same argument/rank, the selected template can be wrong, causing incorrect reshapes and semantic drift.

Change-Id: I9c67bbbb030511a72ec8f4c59b3498ba4dfdb0fc
Signed-off-by: Cathal Corbett <cathal.corbett@arm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants